Adding cURL if no wget on the box#9
Adding cURL if no wget on the box#9roguepullrequest wants to merge 1 commit intoNullArray:masterfrom
Conversation
Added the ability to use cURL if wget is not found on the box. Saw NullArray#5 and agree that that was a poor implementation but this allows for handling of either wget or cURL. If the options for cURL aren't the ones that would give the desired output feel free to change them.
|
Thank you for the PR @roguepullrequest , i am not opposed to including this functionality as a failsafe for when permissions on target don't allow the use of Perhaps in a scenario where you have a limited shell on target which forces you to What would also be a possibility is to encode a static Implementing such a solution would be a matter of adding a conditional to check availability, and invoking a function to perform the operations i just mentioned. in my opinion that would be a more robust solution than simply hoping I would very much like to hear your thoughts on this, before i merge or request changes to the PR to the extent that we have a more robust failsafe mechanism. |
|
This does make sense. If you want to have another fallback option it can be TCP sockets. Let me see what I can cobble together to handle all the scenarios. |
Any updates on any potential improvements you had in mind? |
Added the ability to use cURL if wget is not found on the box. Saw #5 and agree that that was a poor implementation but this allows for handling of either wget or cURL.
If the options for cURL aren't the ones that would give the desired output feel free to change them.